Skip to content

fix: Widget disposal correctly now redirects focus#240

Closed
BenHenning wants to merge 1 commit intoRaspberryPiFoundation:mainfrom
BenHenning:fix-field-edit-focus-behavior
Closed

fix: Widget disposal correctly now redirects focus#240
BenHenning wants to merge 1 commit intoRaspberryPiFoundation:mainfrom
BenHenning:fix-field-edit-focus-behavior

Conversation

@BenHenning
Copy link
Collaborator

@BenHenning BenHenning commented Feb 24, 2025

Fixes #136
Fixes #222

TODO: Fill in description.

WorkspaceSvg.markFocused() only needs to be overwritten in behavior
specifically when closing a widget, not always (to avoid breaking other
Blockly behaviors).
@microbit-matt-hillsdon
Copy link
Contributor

I realise this is in draft, but we took a look while trying to understand everything that might impact user testing. At the moment:

  • this breaks clicking on a block then using keyboard navigation
  • we're worried this approach can only work for known field types e.g. it's currently broken for colour and dropdown in the demo

@BenHenning
Copy link
Collaborator Author

Replaced by #326.

@BenHenning BenHenning closed this Apr 29, 2025
@BenHenning BenHenning deleted the fix-field-edit-focus-behavior branch April 29, 2025 21:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Flyout auto-closes after creating a variable Navigation after field edit broken by recent change

2 participants